Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[cling-cpt] Added indentation and spacing as well as fixing variable references [skip-ci] #10992

Merged
merged 1 commit into from
Jul 28, 2022

Conversation

saisoma123
Copy link
Contributor

This Pull request: Makes the code more flake8 compliant

Changes or fixes: Added indentation and spacing as well as getting rid of unused variables.

Checklist:

  • tested changes locally
  • [NA] updated the docs (if necessary)

This PR fixes issue #406 mentioned in (root-project/cling#406)

@saisoma123 saisoma123 requested a review from vgvassilev as a code owner July 19, 2022 17:41
@phsft-bot
Copy link
Collaborator

Can one of the admins verify this patch?

@lgtm-com
Copy link

lgtm-com bot commented Jul 19, 2022

This pull request fixes 4 alerts when merging 7201488 into 2e38273 - view on LGTM.com

fixed alerts:

  • 3 for Except block handles 'BaseException'
  • 1 for Unused local variable

@vgvassilev vgvassilev merged commit 731f857 into root-project:master Jul 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants